Skip to content

refactor!(storage): Move OpenDal Storage to a new crate iceberg-storage-opendal#2207

Merged
blackmwk merged 9 commits intoapache:mainfrom
CTTY:ctty/storage-crate
Mar 5, 2026
Merged

refactor!(storage): Move OpenDal Storage to a new crate iceberg-storage-opendal#2207
blackmwk merged 9 commits intoapache:mainfrom
CTTY:ctty/storage-crate

Conversation

@CTTY
Copy link
Collaborator

@CTTY CTTY commented Mar 4, 2026

Which issue does this PR close?

What changes are included in this PR?

  • Moved OpenDal Storage impl to a new crate iceberg-storage-opendal(crates/storage/opendal)
  • Renamed feature flags storage-s3, storage-xxx to opendal-s3, opendal-xxx

Are these changes tested?

Relying on the existing uts

@CTTY CTTY added the breaking label Mar 4, 2026
@CTTY CTTY marked this pull request as ready for review March 5, 2026 00:34
Copy link
Contributor

@blackmwk blackmwk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @CTTY for this pr, it looks great! Just some minor comment.

Copy link
Contributor

@blackmwk blackmwk left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @CTTY for this pr!

@blackmwk blackmwk merged commit c408449 into apache:main Mar 5, 2026
20 checks passed
@CTTY CTTY deleted the ctty/storage-crate branch March 5, 2026 18:33
gbrgr pushed a commit to RelationalAI/iceberg-rust that referenced this pull request Mar 10, 2026
…ge-opendal (apache#2207)

<!--
We generally require a GitHub issue to be filed for all bug fixes and
enhancements and this helps us generate change logs for our releases.
You can link an issue to this PR using the GitHub syntax. For example
`Closes apache#123` indicates that this PR will close issue apache#123.
-->

- Closes apache#2209

- Moved OpenDal Storage impl to a new crate
iceberg-storage-opendal(crates/storage/opendal)
- Renamed feature flags `storage-s3`, `storage-xxx` to `opendal-s3`,
`opendal-xxx`

<!--
Provide a summary of the modifications in this PR. List the main changes
such as new features, bug fixes, refactoring, or any other updates.
-->

Relying on the existing uts
<!--
Specify what test covers (unit test, integration test, etc.).

If tests are not included in your PR, please explain why (for example,
are they covered by existing tests)?
-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Move OpenDal Storage impl to a new crate iceberg-storage-opendal

2 participants